projects / llvm-toolchain-13.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 85515b7) | patch
[PATCH] [llvm-objcopy][COFF] Fix test for debug dir presence
authorAlfonso Sánchez-Beato <alfonso.sanchez-beato@canonical.com>
Fri, 10 Sep 2021 08:55:26 +0000 (09:55 +0100)
committerSylvestre Ledru <sylvestre@debian.org>
Fri, 18 Nov 2022 08:47:27 +0000 (08:47 +0000)
commit8bf2b95e56cea347ce37b3a0d953c9176da96e37
tree87e4a9aff941abccbc69beb12c6d255dd81113b5tree | snapshot
parent85515b756ecc51b36260d98458c1b04407a53320commit | diff
[PATCH] [llvm-objcopy][COFF] Fix test for debug dir presence

If the number of directories was 6 (equal to the DEBUG_DIRECTORY
index), patchDebugDirectory() was run even though the debug directory
is actually the 7th entry. Use <= in the comparison to fix that.

This fixes https://llvm.org/PR51243

Differential Revision: https://reviews.llvm.org/D106940

Reviewed by: jhenderson

Gbp-Pq: Name llvm-objcopy-COFF-Fix-test-for-debug-dir-presence.patch
llvm/test/tools/llvm-objcopy/COFF/check-debug-dir-present.test [new file with mode: 0644] blob
llvm/tools/llvm-objcopy/COFF/Writer.cpp diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.